-
Notifications
You must be signed in to change notification settings - Fork 30
Add Product Element #14773
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Product Element #14773
Conversation
… colours for the product cards. Add the Product Card image component. Add the correct sizing to the Picture component for the new Product Image.
# Conflicts: # dotcom-rendering/src/types/content.ts
|
Hello 👋! When you're ready to run Chromatic, please apply the You will need to reapply the label each time you want to run Chromatic. |
Co-authored-by: Jamie B <[email protected]>
Co-authored-by: Jamie B <[email protected]>
JamieB-gu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! One point about star-ratings, but that can be updated later as @oliverabrahams mentions.
|
Seen on PROD (merged by @oliverabrahams 8 minutes and 58 seconds ago) Please check your changes! |
Co-authored-by: Charley_Campbell [email protected]
Co-authored-by: Emma Imber [email protected]
What does this change?
ProductElementcomponent and corresponding storyrenderElementcodecontentWhy?
We have added a new Product element in Composer, which needs to be handled in the rendering layer. The point of this element is to provide a way of structuring product data in Filter articles. This structured data will unlock more design opportunities on Filter pages. Some more details are here if you're interested.
Screenshots
Screenshots are taken using our test article in CAPI CODE
Wide breakpoint
Mobile